Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Optimize OpenSearch Index by Indexing Only Static Metadata Fields #226

Conversation

philippgualdi
Copy link
Collaborator

@philippgualdi philippgualdi commented Jul 9, 2024

Summary

This pull request introduces an optimization to our OpenSearch indexing process by ensuring that only static metadata fields are indexed. This change aims to improve search performance, reduce storage requirements, and streamline the indexing process by excluding dynamic fields that do not contribute to search relevancy.

Background

Previously, our OpenSearch index included both static and dynamic metadata fields. Dynamic fields, while useful for data representation, do not play a significant role in search queries. Indexing these fields unnecessarily increases the index size and impacts search performance.

Changes

Modified the OpenSearch mapping configuration to set dynamic to false for the metadata object. This prevents automatic indexing of any new fields not explicitly defined in the mapping.

Additional Steps

Recreate OpenSearch Index: The OpenSearch index must be recreated to apply the changes effectively.

@philippgualdi philippgualdi force-pushed the modify-index-metadata-fields branch from 2a0a2f6 to 5928dfc Compare July 9, 2024 08:09
@philippgualdi philippgualdi changed the title WIP: Optimize OpenSearch Index by Indexing Only Static Metadata Fields Optimize OpenSearch Index by Indexing Only Static Metadata Fields Jul 9, 2024
@philippgualdi philippgualdi force-pushed the modify-index-metadata-fields branch from 5928dfc to 1eb1f81 Compare July 9, 2024 08:18
@philippgualdi philippgualdi merged commit 485dfe1 into tu-graz-library:master Jul 10, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants